Skip to content

cmake: update standalone example minimum version - #416

Merged
EugenioCollado merged 1 commit into
eProsima:masterfrom
Old-Ding:fix/standalone-cmake-minimum
Jul 6, 2026
Merged

cmake: update standalone example minimum version#416
EugenioCollado merged 1 commit into
eProsima:masterfrom
Old-Ding:fix/standalone-cmake-minimum

Conversation

@Old-Ding

@Old-Ding Old-Ding commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update standalone example/test CMake projects from cmake_minimum_required(VERSION 2.8.12) to 3.5.0
  • keep these standalone files aligned with the other standalone examples/tests that already require CMake 3.5

Why

CMake 4 removed compatibility with projects that request versions older than 3.5, so these standalone CMakeLists fail before configuration can proceed. The top-level project already requires CMake 3.16, and nearby standalone examples/tests in this repo already use 3.5.0.

Testing

  • git diff --check
  • rg -n "cmake_minimum_required\(VERSION 2\.8\.12|cmake_minimum_required\(VERSION 3\.[0-4]" examples test
  • git ls-files --eol $(git diff --name-only HEAD~1..HEAD)

Not run: CMake configure/build, because this local Windows environment does not have CMake or a C/C++ compiler installed.

Generated-by: OpenAI Codex
Signed-off-by: aineoae86-sys <ai.neo.ae86@gmail.com>

@EugenioCollado EugenioCollado left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@EugenioCollado
EugenioCollado merged commit 8360d2c into eProsima:master Jul 6, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants